-
Notifications
You must be signed in to change notification settings - Fork 241
Implement copysign for libcalls #710
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement copysign for libcalls #710
Conversation
compiler-builtins/src/float/div.rs Lines 115 to 116 in 8a610fb
This should get something in |
Also we do have this symbol in I'll still defer to @Amanieu to confirm that though |
The evolution of this is my determination that we don't need it in llvm/llvm-project#111269 |
Is it possible we might need copysign to support gcc-generated code? This may be need for musl targets where we ship a gcc-compiled musl. |
As |
We could probably make adjustments so that we can expose only specific functions from our |
this feels unlikely as there was no situation in which LLVM generated copysign for a type that we also use. if gcc generates that much worse code, then I suggest we file a compiler bug. |
As mentioned elsewhere1, it's suspiciously easy to do this, so why doesn't LLVM handle it?
The answer is it might, in fact, do that2, so accepting this would be a paranoid stopgap at best. Nonetheless, if it pleases you, you may!
Footnotes
https://github.com/rust-lang/rust/pull/131304 ↩
https://github.com/llvm/llvm-project/pull/111269 ↩